Add file check#84
Conversation
There was a problem hiding this comment.
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
src/graphics/vulkan/vk_loader.cpp:210
- [nitpick] Consider using spdlog for logging instead of std::cout to maintain consistency with the rest of the file's logging.
std::cout << "Loading GLTF " << filePath << '\n';
|
ну норм |
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
anunknowperson
left a comment
anunknowperson
left a comment
There was a problem hiding this comment.
Заменить прямой вызов spdlog на макросы которые едины для всего проекта
#include "core/Logging.h"
LOGI(message)
LOGE("{}", message)
anunknowperson
left a comment
anunknowperson
left a comment
There was a problem hiding this comment.
Посмотри есть ли где-то ещё в проекте вызов cout и там замени на LOGI
Фу, макросы в плюсах |
georgiy-belyanin
left a comment
georgiy-belyanin
left a comment
There was a problem hiding this comment.
Thank you for working on it and forming such a nice patch set! LGTM
вроде мы обсуждали макросы и нам разрешили их оставить? не помню. В любом случае, если и менять то это для другой таски. Сейчас во всем проекте юзают макросы. |
Хз, вроде в плюсах неймспейсы для этого всего придуманы. Но в целом, если везде одинаково, то пох. Лучше так, чем везде как попало. |
maybenotilya
left a comment
maybenotilya
left a comment
There was a problem hiding this comment.
Огромное уважение автору данного Pull Request'а за неоценимый вклад в развитие рендерлибы
|
Hey! Any updates on this one? |
1 similar comment
|
Hey! Any updates on this one? |
Добавлена проверка на существование загружаемых файлов в vk_loader.cpp и vk_pipelines.cpp